Skip to content

Fixed unused variable warning, see issue #209 #210

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Feb 7, 2017
Merged

Fixed unused variable warning, see issue #209 #210

merged 1 commit into from
Feb 7, 2017

Conversation

jwillemsen
Copy link
Contributor 

* cores/arduino/wiring.c:

@facchinm facchinm closed this Feb 7, 2017
@jwillemsen
Copy link
Contributor Author

@facchinm For maintainability it looks better to me to move the ul variable to the loop, independent of the fact that this seems to not fix the warning I have.

@cmaglie cmaglie reopened this Feb 7, 2017
@cmaglie cmaglie merged commit 280d92d into arduino:master Feb 7, 2017
@cmaglie cmaglie added this to the Release 1.6.12 milestone Feb 7, 2017
@jwillemsen jwillemsen mentioned this pull request Feb 16, 2017
Closed
boseji pushed a commit to go-ut/combined-ArduinoCore-samd that referenced this pull request May 30, 2020
@hathach
Merge pull request arduino#210 from adafruit/flashdefines
f55ecad 
Flashdefines
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Release 1.6.12
Development

Successfully merging this pull request may close these issues.
3 participants
@jwillemsen @cmaglie @facchinm